Skip to content

Conversation

@YanirShahak
Copy link

As discussed in the previous PR, i've moved this integration into a self-published package.

  • Basic implementation for OCI Generartive AI integration with LangChainJS.
  • Supports invoke and stream.
  • Depends on oci-common and oci-generativeaiinference.
  • Includes two concrete implementations of OciGenAiCohereChat for Cohere models offered in the services as well as OciGenAiGenericChat for all other models.
  • Includes tests & integration tests.
  • Includes documentation.

@vercel
Copy link

vercel bot commented Apr 29, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Project Deployment Preview Comments Updated (UTC)
langchainjs-docs Failed Aug 11, 2025 11:57pm
1 Skipped Deployment
Project Deployment Preview Comments Updated (UTC)
langchainjs-api-refs ⬜️ Ignored Aug 11, 2025 11:57pm

@vercel vercel bot temporarily deployed to Preview – langchainjs-docs April 29, 2025 19:41 Inactive
@YanirShahak
Copy link
Author

@jacoblee93, will you be able to take a peek at this one?

@YanirShahak YanirShahak marked this pull request as ready for review May 20, 2025 17:14
@dosubot dosubot bot added the size:XXL This PR changes 1000+ lines, ignoring generated files. label May 20, 2025
@vercel vercel bot temporarily deployed to Preview – langchainjs-docs May 20, 2025 17:14 Inactive
@YanirShahak
Copy link
Author

YanirShahak commented Aug 6, 2025

@jacoblee93, any interest in merging this?

@vercel vercel bot temporarily deployed to Preview – langchainjs-docs August 6, 2025 17:39 Inactive
Niels Brouwers and others added 2 commits August 6, 2025 17:21
@vercel vercel bot temporarily deployed to Preview – langchainjs-docs August 6, 2025 21:45 Inactive
@vercel vercel bot temporarily deployed to Preview – langchainjs-docs August 11, 2025 23:23 Inactive
@vercel vercel bot temporarily deployed to Preview – langchainjs-docs August 11, 2025 23:57 Inactive
@christian-bromann
Copy link
Member

@YanirShahak thanks for your contribution 🙏 our team has been focused on working on LangChain v1 and unfortunately couldn't give PRs as much attention as they need. We are in an effort to picking up where we left off. I will take this PR and discuss it with the team. Please stay tuned!

@hntrl
Copy link
Member

hntrl commented Sep 11, 2025

@YanirShahak apologies for the delay! When we gave you that direction we more meant that this functionality would be better served in its own repo/ npm package (e.g. langchain-oci-genai instead of within the footprint of the langchain.js monorepo. This is still definitely the case since we're actively trying to reduce the overarching maintenance burden for this repo.

With that in mind, I'm going to close this PR. We're still happy to add integration docs when a separate package is published -- the docs in this PR look very reasonable.

@hntrl hntrl closed this Sep 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants